Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

date_at support #624

Merged
merged 24 commits into from Sep 29, 2014
Merged

date_at support #624

merged 24 commits into from Sep 29, 2014

Conversation

lisps
Copy link
Contributor

@lisps lisps commented Mar 19, 2014

previous PR was #559

@lisps
Copy link
Contributor Author

lisps commented Mar 19, 2014

  • with config date_at_format it is possible to change the date format of at. For example using 2014-03-19-10:28:00
  • if the page has not existed at the given date it now says Page did not exist at %s. It was subsequently created at %s. (dates formatted with date_at_format)
  • @selfthinker the function p_locale_xhtml('showrev'); on the media detail page is used like the one on normal page

@lisps lisps mentioned this pull request Mar 19, 2014
@lisps
Copy link
Contributor Author

lisps commented Apr 8, 2014

what kind of DocUpdate do you meen?

@Klap-in
Copy link
Collaborator

Klap-in commented Apr 8, 2014

All changes that needs an update of the documentation at dokuwiki.org, due to changed or new features, should be tagged with this needDocUpdate. In this case you are adding a new feature.
When a pull request is merged, everyone can find easily the required outstanding doc updates via this tag.

@lisps
Copy link
Contributor Author

lisps commented Apr 8, 2014

ok thanks
is the something else to do/change to get this merged?

@splitbrain
Copy link
Collaborator

two minor things:

  • I don't see a value in having a config variable for the date format used. We could just always use what you used as default - that's sane and good enough for a URL parameter value. I'd even argue we could just always use a timestamp but since we have no UI, yet a humand readable format is a nice gimmik
  • I'm not too found of having another global variable introduced. If it could be avoided it'd be nice but is no big deal

Could you merge master into your branch for easier merging?

@lisps
Copy link
Contributor Author

lisps commented Sep 27, 2014

is this so ok know?

  • i personaly prefer timestamp format, others will requested a readable format - so I am for a config ;)
  • i don't see a way to do it without a global variable, do you?

@splitbrain
Copy link
Collaborator

I really want to get rid of the config option - it will make adding a interface later much harder. how about detecting timestamp or standard date format automatically and support both, but not have a config option?

@lisps
Copy link
Contributor Author

lisps commented Sep 28, 2014

mh, I like the idea .. what about using the strtotime function? It is felxible in detecting time formats even relative ones, and if it fails we assume timestamp or vice versa.

@splitbrain
Copy link
Collaborator

good idea

@scrutinizer-notifier
Copy link

The inspection completed: 1 new issues

splitbrain added a commit that referenced this pull request Sep 29, 2014
@splitbrain splitbrain merged commit 17553fc into dokuwiki:master Sep 29, 2014
@splitbrain
Copy link
Collaborator

@lisps thanks for your work and patience on this 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants